Codecov Report

Merging #23703 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23703   +/-   ##
=======================================
  Coverage   92.24%   92.24%           
=======================================
  Files         161      161           
  Lines       51339    51339           
=======================================
  Hits        47360    47360           
  Misses       3979     3979

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e413c49...4d0e2b3. Read the comment docs.

can you merge master.

@chris-b1 can you take a look here

Hello @gfyoung! Thanks for updating the PR.

• There are no PEP8 issues in the file pandas/io/excel.py !

• There are no PEP8 issues in the file pandas/tests/io/test_excel.py !

@chris-b1 @jreback : Rebased my changes, and all is green. PTAL.

@chris-b1 @jreback : Any further thoughts on this?

@gfyoung it looks fine to me, but @chris-b1 should have a look.

Just to confirm, since they don't diff - none of the existing cases in the xlsx files were modified? Assuming no, looks good to me!

@chris-b1 : I just added a new test to those Excel files. None of the existing tabs were modified. 👍

Alright, with three LGTM's, I think this is good to merge now. 🙂